Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(modal): remove non existant cls-panel option #1014

Merged
merged 1 commit into from
Mar 16, 2023

Conversation

derrabauke
Copy link
Contributor

The clsPanel option got renamed to selPanel which indicates the real usage of it. It's not a css class which will be added for user specifics, but it is a selector for the modal internally. It won't set any classes on the modal element and should be removed from the ember component arguments as well.

Further information:

@derrabauke derrabauke added the dependencies Pull requests that update a dependency file label Mar 16, 2023
@derrabauke derrabauke self-assigned this Mar 16, 2023
The 'clsPanel' option got renamed to 'selPanel' which indicates the
real usage of it. It's not a css class which will be added for user
specifics, but it is a selector for the modal internally. It won't
set any classes on the modal element and should be removed from the
ember component arguments as well.
@derrabauke derrabauke requested a review from anehx March 16, 2023 12:24
@derrabauke derrabauke marked this pull request as ready for review March 16, 2023 12:24
@derrabauke derrabauke requested a review from velrest March 16, 2023 12:25
@velrest
Copy link
Contributor

velrest commented Mar 16, 2023

Looks good but this is a breaking change
Misread it never worked, right?

@derrabauke
Copy link
Contributor Author

Looks good but this is a breaking change Misread it never worked, right?

Since it could not work since ~2019 I guess it's not. 🤷

@velrest velrest merged commit d3252e5 into adfinis:main Mar 16, 2023
@derrabauke derrabauke deleted the fix-tidy-up-modal-options branch March 16, 2023 13:54
anehx pushed a commit that referenced this pull request Mar 28, 2023
# [7.0.0](v6.1.1...v7.0.0) (2023-03-28)

### Bug Fixes

* **modal:** remove non existant cls-panel option ([#1014](#1014)) ([d3252e5](d3252e5))

### chore

* **deps:** update dependencies ([8df899f](8df899f))

### Features

* added danger color to uk-button ([00159e9](00159e9))

### BREAKING CHANGES

* **deps:** Remove support for deprecated Ember LTS 3.24.
@anehx
Copy link
Member

anehx commented Mar 28, 2023

🎉 This PR is included in version 7.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@anehx anehx added the released label Mar 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants